DWS | Task 1 | Ершов Вадим #19
Conversation
![codeclimate[bot]](https://avatars.githubusercontent.com/in/9403?s=60&v=4){kind=small}
src/main/java/ru/vk/itmo/test/vadimershov/utils/MemorySegmentUtil.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/vadimershov/utils/MemorySegmentUtil.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/vadimershov/utils/MemorySegmentUtil.java
Outdated
Show resolved
Hide resolved
| import static ru.vk.itmo.test.vadimershov.utils.MemorySegmentUtil.toEntity; | ||
| import static ru.vk.itmo.test.vadimershov.utils.MemorySegmentUtil.toMemorySegment; | ||
|
|
||
| public class DaoHttpServer extends HttpServer { |
There was a problem hiding this comment.
Не хватает тут обработок неожиданных ошибок, если вдруг упадет ошибка при обработке, то что мы показывали на предыдущем занятии
| Request.METHOD_DELETE | ||
| ); | ||
|
|
||
| private final ReferenceDao dao; |
There was a problem hiding this comment.
В полях лучше использовать как можно более абстрактный класс
| return new Response(Response.NOT_FOUND, Response.EMPTY); | ||
| } | ||
|
|
||
| byte[] value = toByteArray(entry.value()); |
There was a problem hiding this comment.
тут лишнее копирование, которого можно было бы избежать, но пришлось бы попотеть, чтобы сразу отдавать в ответ из мемори сегмента
There was a problem hiding this comment.
я посмотрел как реализован write Response до сокета, как я понимаю можно переопределить так чтоб можно было передать сразу byteBuffer, но тут тоже будет аллокация, или я не в ту сторону думаю?
| } | ||
|
|
||
| @Override | ||
| public CompletableFuture<Void> start() throws IOException { |
There was a problem hiding this comment.
старт и стоп лучше сделать синхронизированными, и сделать защиту всякую, а то если несколько раз вызывать - будут всякие неопределенности
* Task 1 | Implement DaoHttpServer * Task 1 | Fix codeclimate * Task 1 | Fix codeclimate 2 * Task 1 | Add report.md * Task 1 | Fix comments * Task 1 | Fix codeclimate * Task 1 | Fix codeclimate * Task 1 | Fix exception handler * Task 1 | Fix exception handler * Task 1 | Fix exception handler --------- Co-authored-by: ruarsv5 <[email protected]> Co-authored-by: atimofeyev <[email protected]>
* Task 1 | Implement DaoHttpServer * Task 1 | Fix codeclimate * Task 1 | Fix codeclimate 2 * Task 1 | Add report.md * Task 1 | Fix comments * Task 1 | Fix codeclimate * Task 1 | Fix codeclimate * Task 1 | Fix exception handler * Task 1 | Fix exception handler * Task 1 | Fix exception handler --------- Co-authored-by: ruarsv5 <[email protected]> Co-authored-by: atimofeyev <[email protected]>
No description provided.